Skip to content

Conversation

@rogernelson
Copy link
Contributor

Build all python dependencies from source

Summarize your change.

After fixing ABI compatibility issues with OTIO arising from the fact that we are building our own custom python, I checked the other packages and there are indeed others that also have binary components (PyOpenGL_accelerate, pydantic, and cryptography).

Because of this, we should just always build the packages from source with the same compiler we use to build python, and we can do that using the --no-binary option from pip. Although this might be a little bit slower, it will ensure that any package with binary components won't have ABI compatibility issues and for pure python packages it has no effect anyway.

Describe the reason for the change.

ABI safety

Describe what you have tested and on which operating system.

Built on Mac OS 26.1.

Copy link
Contributor

@pbergeron-adsk pbergeron-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cedrik-fuoco-adsk
Copy link
Contributor

Closing in favour of #1014

auto-merge was automatically disabled December 17, 2025 13:32

Pull request was closed

cedrik-fuoco-adsk added a commit that referenced this pull request Dec 18, 2025
)

###

### Linked issues
n/a

### Summarize your change.
Install and set-up dependencies for Python/Pip to build its dependencies
from source.

### Describe the reason for the change.
Follow up on
#1013 to install
the needed dependencies to build Numpy and Cryptography from source.

### Describe what you have tested and on which operating system.
CI

### Add a list of changes, and note any that might need special
attention during the review.

### If possible, provide screenshots.

---------

Signed-off-by: Roger Nelson <[email protected]>
Signed-off-by: Cédrik Fuoco <[email protected]>
Co-authored-by: Roger Nelson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants